Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Remove fluidity from body text and spacing in Compact Mode #1389

Merged

Conversation

VincentSmedinga
Copy link
Contributor

As discussed, having fluid type in Compact Mode makes less sense now that regular text only grows 2 pixels between window widths of 320 and 1600, which comes down to 0.15%. This PR fixates Text Level 5 to 16 pixels.

This required the sizes to grow a bit slower on narrow screens. I’ve tested it with Amopis, looks good to me.

Because space lengths are based on Text Level 5, this also is no longer fluid in Compact Mode.

@github-actions github-actions bot temporarily deployed to demo-DES-882-3-fixate-compact-text-level-5 July 16, 2024 12:16 Destroyed
@github-actions github-actions bot temporarily deployed to demo-DES-882-3-fixate-compact-text-level-5 July 16, 2024 13:52 Destroyed
@VincentSmedinga VincentSmedinga requested a review from dlnr July 16, 2024 16:05
Copy link
Contributor

@dlnr dlnr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Personally I would like to see what this does to mockups in Figma so let do this and find out.

@github-actions github-actions bot temporarily deployed to demo-DES-882-3-fixate-compact-text-level-5 July 17, 2024 12:09 Destroyed
@VincentSmedinga VincentSmedinga changed the title feat: Remove fluidity from text and spacing in Compact Mode feat: Remove fluidity from body text and spacing in Compact Mode Jul 17, 2024
@VincentSmedinga VincentSmedinga merged commit 050b063 into develop Jul 18, 2024
8 checks passed
@VincentSmedinga VincentSmedinga deleted the feature/DES-882-3-fixate-compact-text-level-5 branch July 18, 2024 09:12
@github-actions github-actions bot mentioned this pull request Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants